integrate dsander's suggestions
Andrew Cantino
2 fichiers modifiés avec 6 ajouts et 2 suppressions
+ 5
- 1
app/controllers/agents_controller.rb
@@ -98,7 +98,11 @@ class AgentsController < ApplicationController |
||
| 98 | 98 |
end |
| 99 | 99 |
|
| 100 | 100 |
def diagram |
| 101 |
- @agents = ((params[:scenario_id].present? && current_user.scenarios.find(params[:scenario_id])) || current_user).agents.includes(:receivers) |
|
| 101 |
+ @agents = if params[:scenario_id].present? |
|
| 102 |
+ current_user.scenarios.find(params[:scenario_id]).agents.includes(:receivers) |
|
| 103 |
+ else |
|
| 104 |
+ current_user.agents.includes(:receivers) |
|
| 105 |
+ end |
|
| 102 | 106 |
end |
| 103 | 107 |
|
| 104 | 108 |
def create |
+ 1
- 1
app/views/scenarios/index.html.erb
@@ -20,7 +20,7 @@ |
||
| 20 | 20 |
<% @scenarios.each do |scenario| %> |
| 21 | 21 |
<tr> |
| 22 | 22 |
<td> |
| 23 |
- <%= link_to(scenario.name, scenario, class: "label label-info") %> |
|
| 23 |
+ <%= link_to(scenario.name, scenario) %> |
|
| 24 | 24 |
</td> |
| 25 | 25 |
<td><%= link_to pluralize(scenario.agents.count, "agent"), scenario %></td> |
| 26 | 26 |
<td><%= scenario.public? ? "yes" : "no" %></td> |